inspector: Stop using ::key-press-event

testgtk: Drop snapshot

This needs to be redone with modern snapshotting infrastructure.
Lets drop it for now, it is somewhat duplicative with inspector
features.

testgtk: Drop the event watcher

This is not a very useful test, and it is hard to keep working
in an event controller world.

docs: Modernize an example

The signals that are showcased here are going away.

gtk-demo: Stop using ::key-press-event in font explorer

Use a key controller instead. Note that this currently
breaks the handling of Enter, since we activate key bindings
twice, causing us to switch to the label and back.

testsuite: Don't use ::key-press-event

We don't need it here.

file chooser: Use a key event controller

We want to get rid of ::key-press-event.

demos: Add "Paint" demo

gtk: Add GtkGestureStylus

This is a GtkGesture done to deal with stylus events from drawing tablets.
Those have a special number of characteristics that extend a regular
pointer, so it makes sense to wrap that.

gtkwindow: Unset focus grab_widget if it ends up unmapped

This may result on the later emission of crossing events, with one of the
sides being already unmapped/unrealized. The widget being unmapped will
result on repick and emission of a set of crossing events anyway.

gtk: Add GtkEventControllerKey

This event controller is meant to replace usage from key-press/release-event
handlers all through. Optionally it can be set a GtkIMContext, so interaction
is carried by the controller.

snapshot: Don't cause invalid reads

1. Pass clip rectangles to gtk_snapshot_push_state() that point into
   the state array.
2. g_array_set_size(len+1) the state array
3. Make that function realloc() the state array.
4. The clip rectangle now points into invalid memory
5. Use the clip array

This patch fixes things by moving step 5 to before step 2.

rendernode: Add missing return

Don't to extra work when not needed.

Update POTFILES.in

widget: Fix variable names

Stop docs build from complaining

a11y: Fix headers

We were getting a gcc warning before.

Merge branch 'window-activate' into 'master'

a11y: restore window:activate/deactivate emission

Closes #127

See merge request GNOME/gtk!77

a11y: restore window:activate/deactivate emission

1b9aa1b708c2 ('a11y: drop the focus tracker') removed a bit too much.  We
still have to emit window:activate/deactivate events.  They are easy to
emit anyway.

Fixes #127

inspector: Add an fps overlay

inspector: Bring back debug updates

This time, they are implemented as an overlay, so they require a running
inspector and can't be enabled via env variable anymore.

widget: Remove draw signal

The only remaining user is the text handle, but the text handle doesn't
draw anything anyway currently, so whoever fixes it can make it use
snapshots.

inspector: Introduce so-called "overlays"

Overlays are drawings that get rendered on top of the inspected window.

The only overlay in existence so far is the highlight overlay, which is
used to highlight widgets and replaces the "draw" signal handler used
previously.

widget: Add gtk_widget_compute_bounds()

The first in a set of functions intended to query widget coordinates
from another widget's coordinate system.

inspector: Change semantics of inspector render node notify

Instead of just notifying the inspector of what is going to be rendered,
allow the inspector to modify it.

This way, the inspector can overlay information it deems relevant over
the render node while still having access to what the actual widget
(without the inspector) would paint.

image: Actually report a request mode

This makes height-for-width actually work for GtkImage.

I'm kinda ashamed I didn't notice it not working before.

treeview: Remove invalidation tracking code

testgtk: Remove draw signal usage

widget: Remove GtkWidget.draw vfunc

Widgets are exclusively snapshot now.

The draw signal still exists.

widget: Remove gtk_widget_draw()

If you want to draw a widget to cairo today, you create a widget
paintable, snapshot it to a render node and then draw the render node to
cairo.

And yes, this is that complicated on purpose. Don't draw widgets to
Cairo.

reftests: Port from gtk_widget_draw() to snapshots

testgtk: Take an actual snapshot of a widget

Don't just draw the widget, take the paintable of it instead!

testlist3: Set the actual row as drag icon

(Well, don't do anything actually, because dnd is broken, but
in theory...)

widgetpaintable: Implement get_current_image()

Now that snapshot and empty paintables exist, this is rather trivial.

paintable: Introduce gdk_paintable_new_empty()

Also, use it where appropriate.

snapshot: Allow passing the bounds of the created paintable

This allows being more specific about the size.
It's useful in particular when the resulting render nodes might be
too small for the size, not only when they are too large. For the
latter case, using a clip node would be enough.

It also requires adding a clip node when rendering the resulting
paintable, but that should be optimized out by GtkSnapshot when not
necessary.

widgetpaintable: Protect against too many signals

This is actually not just a mechnaism to protect against too many
signals, but it's also a method to getting those signals at the wrong
time.

For every size/content change, a widget needs to invalidate twice:
Once when it queues a resize/redraw (going valid => invalid) and once
when the new size/content is actually assigned (going invalid => valid).

However, one of those invalidations might be inconvenient for the
listener. GtkImage for example does not like receiving
invalidate-contents signals when new contents are assigned, but is fine
with them when the old ones go invalid. And it will not try to draw the
paintable in between anyway.

So by bypassing the 2nd emission if nothing was changed, we can make
GtkImage happy.

demos: Add an image demo for widget paintables

widget: Remove clip from size-allocate vfunc

As the clip is no longer needed, get rid of it.

widget: Don't keep track of clip anymore.

It's not used anywhere, so why would we need it?

broadway: Improve error message

This error message is printed into the journal if a GTK app can't
connect to eithre Wayland or X11. Make it at least mention who is not
capable of connecting to a server.

Unrelated, we might want to improve our error reporting when a GTK app
can't start, so that debugging issues with system startup / login get
easier to resolve.

widget: Don't clip widgets anymore when snapshotting

This is the first step in removing clips.

It's seperate to allow bisecting to this commit when analyzing
performance problems.

widget: Cache clip from creating render node

When the clip changes that is passed to a snapshot function, we need to
create eventual cached render nodes because they might not have drawn
their whole area before.

Fixes issues with redrawing when scrolling.

snapshot: Track clip as a rect only

diff: Allow aborting a diff

When the max cost for finding a path gets to high, the diff can now be
aborted.

Because render nodes have a fallback method (by just marking the whole
bounds of the nodes as different), we use this to improve performance
of diffs.

This brings fishbowl (which is basically a container node with N images
that change every frame) back to close to previous performance.

build: Switch order of subdirectories

After fixing this once already in 5970dac1679d the order was messed up
again in d50708b0240.

diff: Introduce GskDiffSettings

We can put lots of settings there to allow tuning the diff algorithm
used.

vulkan: Always render clip extents

Now that we have the full render nodes available, there is not much
benefit in fine-grained control over multiple rectangles.
In particular, it's causing pain with complex regions.

There might be a benefit in clipping to the region's rectangles in cases
like widget-factory where the whole diff is made up of the 2 rectangles
of spinner and the pulsing progress bar, but it needs a good heuristic
for where this is useful.

snapshot: Remove clip argument from gtk_snapshot_new()

It's not used anymore. And anybody who wants to have a clip on a
newly created snapshot can achieve that using gtk_snapshot_push_clip().

gtk: Remove gtk_widget_queue_draw_region()

... and gtk_widget_queue_draw_area().

They don't doi anything anymore.

widget: Stop tracking invalidations

Instead of calling gdk_surface_invalidate_region(), just
gdk_surface_queue_expose() and rely on the renderer computing the diff
from the previous rendering.

gtk: Remove debug updates

With the soon-to-arrive automatic updates (aka computing invalid region
from render nodes), this will no longer be necessary.

renderer: Track the previous node

... and diff the previous node with the current one to determine the
clip region.

This doubles the work necessary to track clip regions, but the following
commits will clean that up.

gsk: move begin/end_frame vfuncs into the renderers

gsk: Fold gsk_renderer_begin_draw_frame() into renderer

It doesn't need to be exported anymore.

As a side effect, the inspector no longer has any information about the
render region, so remove the code that was taking care of that.

widget: Reorder gtk_widget_render()

Now that we don't clip the created render nodes anymore, we don't have
to compute the clip region beforehand.

So snapshot the render nodes before initializing the renderer.

gsk: Implement gsk_render_node_diff()

This includes a copy of the diff(1) algorithm used by git diff by Davide
Libenzi.

It's used for the common case ofcontainer nodes having only very few
changes for the few nodes of child widgets that changed (like a button
lighting up when hilighted or a spinning spinner).

magnifier: Redo with GtkWidgetPaintable

gtk: Add GtkWidgetPaintable

A GtkWidgetPaintable is a paintable that observes a given GtkWidget and
renders that widget into a paintable.

widget: Cache the render node

This requires a bunch of refactorings:

1. Don't pass the current clip region to gtk_widget_snapshot()
   so we don't create full widget contents
3. Have a widget->priv->draw_needed that we invalidate on every
   queue_draw() call and set on every snapshot()
2. In queue_draw(), walk the widget chain to invalidate the
   render nodes of all parents

rendernode: Add gsk_render_node_diff()

... and gsk_render_node_can_diff(). Those are vfuncs to compute a region
containing all the pixels that differ between the two nodes.

This is just the plumbing that chains into node classes. No node
implements it yet.

font button: fix compiler warnings

A PangoLanguage is not a string.

Merge branch 'text-changed-delete-4' into 'master'

gtkentryaccessible: Fix text-changed::delete length

See merge request GNOME/gtk!100

Merge branch 'textview-delete-4' into 'master'

gtktextviewaccessible: update cursor position after text suppression

See merge request GNOME/gtk!102

gtktextviewaccessible: update cursor position after text suppression

delete_range_cb is set to be called before the text suppression done by
the gtktextlayout (otherwise it does not work properly). But at that
point the cursor position is not yet up to date.  We thus need to move
the accessibility cursor notification to after the actual text
suppression, by using another callback.

This fixes cursor position in brltty screen reading.

(cherry picked from commit fa6994d0331a219e85dce01007b2d6f3f7fd43c9)

gtkentryaccessible: Fix text-changed::delete length

The second parameter of the text-changed::delete event is to be the length,
not the end position.  This fixes spurious text removals in brltty
screen reading.

(cherry picked from commit 209f908a03a9d27508704425a57dcd533fab4c7d)

GLArea: Remove wrong transfer annotation on new()

Like other widgets, this returns a floating reference, so
(transfer full) is wrong. Just omit the annotation as others do,
thus implying (transfer none).

Close https://gitlab.gnome.org/GNOME/gtk/issues/156

imwayland: destroy objects only if it matches our manager object

Should fix one of the crash from #129.

font chooser: Remove some dead code

This is leftover code from already removed debug spew.

font button: Fix a crash

Since we are now storing the language as a PangoLanguage,
we must not free it anymore.

font chooser: Make the language property writable

This is meant as an input to the font chooser.
We don't want the user to select a language, but
rather have fonts presented as they would work for
the current language. Therefore, do away with the
lang/script combo on the tweak page.

gtk, demos: Fix builds without HarfBuzz and PangoFT

Really exclude the portions in the gtkfontchooserwidget.c that are built
when HarfBuzz and PangoFT2 are built, and update the Meson files to
exclude such sources as well from the main GTK SO/DLL and from the
gtk4-demo program.

https://bugzilla.gnome.org/show_bug.cgi?id=773299

testsuite/gsk/test-render-nodes.c: Avoid VLA usage

Just #define N as 5, instead of using a const int, which is not enough
to justify that as non-VLA usage.

https://bugzilla.gnome.org/show_bug.cgi?id=773299

[gtk|demos/gtk-demo]/language-names.c: Fix build on non-GCC/CLang

Remove g_auto*() usage from these sources and use the traditional
g_free(), as g_auto*() are GCCisms (or CLangisms).

Also, don't include unistd.h unconditionally and stop including
langinfo.h and dirent.h, since they seem to be unused.

https://bugzilla.gnome.org/show_bug.cgi?id=773299

build: Fix linking demos on Visual Studio

The demos are now built as GUI programs, which will require the presence
of WinMain() on Visual Studio builds, unless we specify the entry point.

Pass the /entry:mainCRTStartup linker flag on Visual Studio builds for
the demo programs so that they can link properly.

https://bugzilla.gnome.org/show_bug.cgi?id=773299

font chooser: Add examples for font features

For some font features, we can figure out affected
glyphs, and show before/after. For some others, we
hardcode typical sequences.

Still to do: figure out how to find ligatures and
show them.

widget-factory: Enable all details in the font chooser

We want to see this here.

Update POTFILES.in

W32: Link GTK to pangowin32

Needed for pango_win32_font_logfont() from gtkimcontextime.

https://bugzilla.gnome.org/show_bug.cgi?id=773299

Merge branch 'lrn/issue-147' into 'master'

Fix cursor handling in GTK4 on W32

Closes #147

See merge request GNOME/gtk!92

Fix some g-i annotation warnings

Enable variations and features for font explorer

font chooser: Align level value names

In the backport, I used the plural for variations, and that
is a better name, so use it here too.

font chooser: Align property defaults

Set the default font chooser level to the same value we use
in the backports, to preserve font chooser behavior.

Font chooser: Hide tweak button sometimes

When there is no chance of tweaking anything, don't
show a tweak button.

Defeat compiler optimization

We always want to call both update functions here, and
the compiler unhelpfully optimizes out the second call
if the first one returns TRUE.

GskRenderer: Remove outdated documentation comment

All render node trees are immutable

snapshot: Plug memory leak when collecting rounded clip nodes

gl renderer: Don't highlight cairo nodes as fallback

ProgressBar: Drop leftover "superimposed" mention

This was missed from the commit that dropped other mentions to the
previous ability to superimpose text.

https://bugzilla.gnome.org/show_bug.cgi?id=748784

appchooserwidget: search text using g_str_match_string()

so we match on diacritics (eg. accented letters) and also
search on text tokens. As a plus we also reduce code size.

https://bugzilla.gnome.org/show_bug.cgi?id=745128

togglebutton: Remove priv pointer

popover: Remove priv pointer

shaderbuilder: Insert newlines directly after defines

Instead of doing that in the loop in the last interation, we can just do
it after the loop.

gl renderer: Remover unused mask uniform

shader builder: Remove unused members

shader builder: Reuse GString for shader code

widget: Fix snapshot condition

Of course, we only need to snapshot if both width and height > 0,
otherwise the result is invisible.

entrybuffer: Remove priv pointer

menubar: Don't crash when cycling through menubars

Merge branch 'im-wayland-reset' into 'master'

wayland: Don't emit signals if nothing changed

See merge request GNOME/gtk!83